Skip to content

fix(nospace): don't move to next word when removing last character (@Leonabcd123)#8038

Merged
Miodec merged 3 commits into
monkeytypegame:masterfrom
Leonabcd123:fix/nospace-word-navigation
Jun 2, 2026
Merged

fix(nospace): don't move to next word when removing last character (@Leonabcd123)#8038
Miodec merged 3 commits into
monkeytypegame:masterfrom
Leonabcd123:fix/nospace-word-navigation

Conversation

@Leonabcd123
Copy link
Copy Markdown
Contributor

@Leonabcd123 Leonabcd123 commented May 31, 2026

Fixes #8037

Not sure whether stop on error = word should have any effect when nospace is enabled, so this pr currently only fixes the case where a mistake isn't blocked at the end of a word in nospace.

@monkeytypegeorge monkeytypegeorge added the frontend User interface or web stuff label May 31, 2026
@Leonabcd123 Leonabcd123 marked this pull request as ready for review May 31, 2026 18:25
@github-actions github-actions Bot added the waiting for review Pull requests that require a review before continuing label May 31, 2026
@Miodec
Copy link
Copy Markdown
Member

Miodec commented Jun 1, 2026

Is the reorder necessary?

@Miodec Miodec added waiting for update Pull requests or issues that require changes/comments before continuing and removed waiting for review Pull requests that require a review before continuing labels Jun 1, 2026
@Leonabcd123
Copy link
Copy Markdown
Contributor Author

Leonabcd123 commented Jun 2, 2026

Is the reorder necessary?

Yes, shouldGoToNextWord is now dependent on removeLastChar, and shouldGoToNextWord is used before removeLastChar is declared (on master). Without reordering,

if (!shouldGoToNextWord) {
TestInput.corrected.update(data, correct);
}

Could be skipped when shouldGoToNextWord = true and removeLastChar = true, even though we aren't moving to the next word, so it should run.

@github-actions github-actions Bot removed the waiting for update Pull requests or issues that require changes/comments before continuing label Jun 2, 2026
@Miodec Miodec merged commit 7ccdea5 into monkeytypegame:master Jun 2, 2026
15 checks passed
@Leonabcd123 Leonabcd123 deleted the fix/nospace-word-navigation branch June 2, 2026 13:14
@elfi-ox
Copy link
Copy Markdown

elfi-ox commented Jun 2, 2026

Thanks for fixing this so rapidly. Your contribution is greatly appreciated ❤️

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

frontend User interface or web stuff

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Weird edge cases when combining the "Stop on error" option and the funbox mode "nospace"

4 participants